test(ci): pin RHDH 1.10 RC image (1.10-128) in showcase value files - #4903
test(ci): pin RHDH 1.10 RC image (1.10-128) in showcase value files#4903gustavolira wants to merge 2 commits into
Conversation
…skip-build] Point values_showcase.yaml and values_showcase-rbac.yaml at the RHDH 1.10 RC image quay.io/rhdh/rhdh-hub-rhel9:1.10-128 for release-1.10 e2e testing. Note: CI may override the image via --set upstream.backstage.image.* (IMAGE_REGISTRY/IMAGE_REPO/TAG_NAME) depending on the job. Co-Authored-By: Claude Opus 4.8 (1M context) <noreply@anthropic.com>
Code Review by Qodo
1. Pinned image never applied
|
Review Summary by QodoPin RHDH 1.10 RC image in showcase Helm value files
WalkthroughsDescription• Pin RHDH 1.10 RC image (1.10-128) in showcase value files • Update both showcase and showcase-rbac Helm value files • Set explicit image registry, repository, and tag values • Add clarifying comments about CI override behavior Diagramflowchart LR
A["Showcase Value Files"] -->|"Pin image to"| B["quay.io/rhdh/rhdh-hub-rhel9:1.10-128"]
B -->|"For release-1.10"| C["E2E Testing"]
D["CI --set flags"] -.->|"May override"| B
File Changes1. .ci/pipelines/value_files/values_showcase.yaml
|
…kip-build] helm::get_image_params() injects --set upstream.backstage.image.* from IMAGE_REGISTRY/IMAGE_REPO/TAG_NAME, which overrides the value files. Force these to quay.io/rhdh/rhdh-hub-rhel9:1.10-128 so release-1.10 e2e jobs deploy the RC image regardless of what Prow provides. Temporary; revert after RC testing. Co-Authored-By: Claude Opus 4.8 (1M context) <noreply@anthropic.com>
|
|
/test ? |
|
/test e2e-ocp-helm-nightly |
|
@gustavolira: The following test failed, say
Full PR test history. Your PR dashboard. DetailsInstructions for interacting with me using PR comments are available here. If you have questions or suggestions related to my behavior, please file an issue against the kubernetes-sigs/prow repository. I understand the commands that are listed here. |



Summary
Force the
release-1.10e2e jobs to deploy the RHDH 1.10 RC imagequay.io/rhdh/rhdh-hub-rhel9:1.10-128(digestsha256:943af223a509add22de5c2da2803d363f0219eabaa96df945bb11c85f851b920).Changes
.ci/pipelines/env_variables.sh— forceIMAGE_REGISTRY/IMAGE_REPO/TAG_NAMEto the RC image. This is the change that actually takes effect, becausehelm::get_image_params()(.ci/pipelines/lib/helm.sh) injects--set upstream.backstage.image.{registry,repository,tag}from these vars, and--setoverrides value files. Marked TEMPORARY — revert after RC testing..ci/pipelines/value_files/values_showcase.yamlandvalues_showcase-rbac.yaml— also pinupstream.backstage.imageto the RC image, for documentation/consistency (effective only when CI does not pass image--setflags).Notes
[skip-build]so Prow skips rebuilding the image and the e2e jobs run against the pre-built RC image.release-1.10; theenv_variables.shoverride block should be reverted once validation is complete.🤖 Generated with Claude Code